Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: update coverage configuration for SonarCloud integration #293

Conversation

danstis
Copy link
Owner

@danstis danstis commented Oct 5, 2024

User description

Fixes #291


PR Type

configuration changes


Description

  • Added relative_files = True to the [coverage:run] section in tox.ini to fix the issue with SonarCloud coverage reporting.
  • This change ensures that coverage paths are reported relative to the source, which is necessary for correct integration with SonarCloud.

Changes walkthrough 📝

Relevant files
Configuration changes
tox.ini
Update coverage configuration to use relative file paths 

tox.ini

  • Added relative_files = True under [coverage:run] section.
  • This change affects how coverage paths are reported.
  • +1/-0     

    💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

    @sophie-syntax sophie-syntax bot changed the title ci: fix issue with SonarCloud coverage on build pipeline ci: update coverage configuration for SonarCloud integration Oct 5, 2024
    Copy link

    sophie-syntax bot commented Oct 5, 2024

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    tox.ini Show resolved Hide resolved
    Copy link

    sonarcloud bot commented Oct 5, 2024

    @danstis danstis merged commit bdb5779 into main Oct 5, 2024
    8 checks passed
    @danstis danstis deleted the danstis/291-Fix-issue-with-SonarCloud-coverage-on-build-pipeline branch October 5, 2024 03:25
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    Fix issue with SonarCloud coverage on build pipeline
    1 participant